Skip to content

Conversation

@JavierGarciadeCastro
Copy link

PR description:
This is a backport of PR #47938. This PR is designed for CMSSW_15_0_X to run the new DQM in the data-taking release, whereas the original (#47938) was designed for CMSSW_15_1_X. Description is specified in original PR.
PR validation:
We have checked out all dependencies and have a clean build. We applied the code-checks specified in PR #47235. Most runTheMatrix tests were successfully run as well, with a couple of errors coming from files not found:
runall-report-step123-.log

@cmsbuild
Copy link
Contributor

cmsbuild commented Apr 30, 2025

A new Pull Request was created by @JavierGarciadeCastro for CMSSW_15_0_X.

It involves the following packages:

  • DQMOffline/HLTScouting (dqm)
  • HLTriggerOffline/Scouting (dqm)

@antoniovagnerini, @cmsbuild, @rseidita can you please review it and eventually sign? Thanks.
@missirol, @mmusich, @mtosi, @rociovilar this is something you requested to watch as well.
@antoniovilela, @mandrenguyen, @rappoccio, @sextonkennedy you are the release manager for this.

cms-bot commands are listed here

@cmsbuild
Copy link
Contributor

cmsbuild commented Apr 30, 2025

cms-bot internal usage

@silviodonato
Copy link
Contributor

please test

@mmusich
Copy link
Contributor

mmusich commented May 5, 2025

backport of #47938

@mmusich
Copy link
Contributor

mmusich commented May 5, 2025

@cmsbuild, please abort

@mmusich
Copy link
Contributor

mmusich commented May 5, 2025

test parameters:

  • workflow = 145.415

@mmusich
Copy link
Contributor

mmusich commented May 5, 2025

@cmsbuild, please test

@cmsbuild
Copy link
Contributor

cmsbuild commented May 5, 2025

+1

Size: This PR adds an extra 16KB to repository
Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-237947/45837/summary.html
COMMIT: a9ea821
CMSSW: CMSSW_15_0_X_2025-05-05-1100/el8_amd64_gcc12
User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week0/cms-sw/cmssw/47992/45837/install.sh to create a dev area with all the needed externals and cmssw changes.

Comparison Summary

Summary:

  • You potentially added 2 lines to the logs
  • Reco comparison results: 2 differences found in the comparisons
  • DQMHistoTests: Total files compared: 51
  • DQMHistoTests: Total histograms compared: 4105704
  • DQMHistoTests: Total failures: 98
  • DQMHistoTests: Total nulls: 1
  • DQMHistoTests: Total successes: 4105585
  • DQMHistoTests: Total skipped: 20
  • DQMHistoTests: Total Missing objects: 0
  • DQMHistoSizes: Histogram memory added: 64.409 KiB( 50 files compared)
  • DQMHistoSizes: changed ( 145.415 ): 64.409 KiB HLT/ScoutingOffline
  • Checked 221 log files, 193 edm output root files, 51 DQM output files
  • TriggerResults: no differences found

@silviodonato
Copy link
Contributor

@cms-sw/dqm-l2 do you have any comments?

@silviodonato
Copy link
Contributor

Hi @antoniovilela @mandrenguyen ,
we (Scouting/TSG) would like to have these three PRs #47992 #47941 #47935 included in the next 15_0_X release.
They are all updates/fixes to the scouting offline DQM and I believe they can be safety merged.

@antoniovagnerini
Copy link

+dqm

@cmsbuild
Copy link
Contributor

Pull request #47992 was updated. @antoniovagnerini, @cmsbuild, @ctarricone, @rseidita can you please check and sign again.

@mmusich
Copy link
Contributor

mmusich commented May 14, 2025

@cmsbuild, please test

@mandrenguyen
Copy link
Contributor

unhold

@cmsbuild cmsbuild removed the hold label May 14, 2025
@cmsbuild
Copy link
Contributor

+1

Size: This PR adds an extra 20KB to repository
Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-237947/46137/summary.html
COMMIT: fa80ed8
CMSSW: CMSSW_15_0_X_2025-05-14-1100/el8_amd64_gcc12
User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week1/cms-sw/cmssw/47992/46137/install.sh to create a dev area with all the needed externals and cmssw changes.

Comparison Summary

Summary:

  • You potentially removed 2 lines from the logs
  • Reco comparison results: 11 differences found in the comparisons
  • DQMHistoTests: Total files compared: 51
  • DQMHistoTests: Total histograms compared: 4098938
  • DQMHistoTests: Total failures: 39
  • DQMHistoTests: Total nulls: 1
  • DQMHistoTests: Total successes: 4098878
  • DQMHistoTests: Total skipped: 20
  • DQMHistoTests: Total Missing objects: 0
  • DQMHistoSizes: Histogram memory added: 64.409 KiB( 50 files compared)
  • DQMHistoSizes: changed ( 145.415 ): 64.409 KiB HLT/ScoutingOffline
  • Checked 221 log files, 193 edm output root files, 51 DQM output files
  • TriggerResults: no differences found

@mandrenguyen
Copy link
Contributor

+1

@mandrenguyen
Copy link
Contributor

merge

@cmsbuild cmsbuild merged commit c62b73b into cms-sw:CMSSW_15_0_X May 14, 2025
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants